Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CI/CD using Github Actions for linting and building the Django app #7

Closed
wants to merge 4 commits into from

Conversation

sweep-ai[bot]
Copy link
Contributor

@sweep-ai sweep-ai bot commented Oct 1, 2023

PR Feedback (click)

  • 👍 Sweep Did Well
  • 👎 Sweep Needs Improvement

Description

This PR adds a CI/CD pipeline using Github Actions for linting (using flake8) and building the Django app. It also implements a pre-commit approach for linting and includes a step to push the Docker image to DockerHub.

Summary of Changes

  • Added a Github Actions workflow file (.github/workflows/ci_cd.yml) to define the CI/CD pipeline.
  • Installed flake8 and added a step to run flake8 in the workflow for linting.
  • Added a step to build the Django app in the workflow.
  • Added a step to push the Docker image to DockerHub in the workflow.
  • Added a pre-commit hook for flake8 by creating a .pre-commit-config.yaml file.
  • Modified the requirements.txt file to include flake8 as a requirement.
  • Modified the my_site/Dockerfile to include steps for copying the .pre-commit-config.yaml file and installing pre-commit.

Please review and merge this PR to enable CI/CD for the Djangoline repository.

Fixes #6.


🎉 Latest improvements to Sweep:


💡 To get Sweep to edit this pull request, you can:

  • Leave a comment below to get Sweep to edit the entire PR
  • Leave a comment in the code will only modify the file
  • Edit the original issue to get Sweep to recreate the PR from scratch

@sweep-ai sweep-ai bot added the sweep Assigns Sweep to an issue or pull request. label Oct 1, 2023
@sweep-ai sweep-ai bot closed this Oct 1, 2023
@sweep-ai sweep-ai bot deleted the sweep/add-ci-cd branch October 1, 2023 23:17
Copy link
Owner

@s403o s403o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweep: fix building step

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sweep: make a CI\CD using Github Actions for linting (using flake8) and building the Django app
1 participant